Skip to content

feat(assets): 使用 npm 管理前端外部依赖#212

Merged
ThaddeusJiang merged 3 commits into
mainfrom
feat/npm-frontend-assets
Jun 13, 2026
Merged

feat(assets): 使用 npm 管理前端外部依赖#212
ThaddeusJiang merged 3 commits into
mainfrom
feat/npm-frontend-assets

Conversation

@ThaddeusJiang

@ThaddeusJiang ThaddeusJiang commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • 将 Tailwind CSS、daisyUI、Heroicons、Choices.js 改为通过 assets/package.jsonassets/package-lock.json 管理。
  • 移除 assets/vendor 中复制的外部依赖源码,并更新 CSS/JS import。
  • 保留 Phoenix JS packages 与 esbuild 在 Mix/Hex 依赖中管理。
  • 更新 Docker 构建、开发文档和资产管线测试。
  • 已合并最新 main,保留 README / self-hosting 文档结构更新,并解决 Docker 文档冲突。

Why

  • 避免继续复制外部依赖源码到仓库,统一使用 npm lockfile 管理外部前端包。
  • 保持 Phoenix/LiveView/esbuild 仍沿用 Phoenix/Mix 生态,减少重复依赖管理。
  • 保留 main 对安装文档和 self-hosting 文档入口的整理。

Validation

  1. MIX_DEPS_PATH=deps mix format
  2. MIX_DEPS_PATH=deps mix compile --warnings-as-errors
  3. DATABASE_URL=postgres://postgres:postgres@localhost:20001/vmemo_test TYPESENSE_URL=http://localhost:20002 TYPESENSE_API_KEY=xyz MIX_DEPS_PATH=deps mix test test/vmemo/assets_pipeline_test.exs
  4. npm ci --prefix assets
  5. npm run build --prefix assets
  6. MIX_DEPS_PATH=deps mix assets.deploy
  7. rg -n '^<<<<<<<|^=======|^>>>>>>>' .
  8. git diff --check

Related

  • Issues: N/A
  • PRs: N/A

Notes

  • phoenixphoenix_htmlphoenix_live_view 不进入 npm。
  • esbuild 不进入 npm,继续使用 Mix/Hex wrapper。
  • config :tailwind 继续移除,CSS 由 npm Tailwind CLI 构建。
  • Docker maintainer doc 遵循 main 的精简结构;npm builder 约束保留在 Canonical Policy / Startup Checklist。

@ThaddeusJiang ThaddeusJiang self-assigned this Jun 12, 2026
@ThaddeusJiang

Copy link
Copy Markdown
Owner Author

已合并最新 main 并解决冲突;PR 描述已同步。

@ThaddeusJiang ThaddeusJiang changed the title feat(assets): 使用 npm 管理前端外部依赖 chore(assets): use NPM Jun 13, 2026
@ThaddeusJiang ThaddeusJiang changed the title chore(assets): use NPM feat(assets): 使用 npm 管理前端外部依赖 Jun 13, 2026
@ThaddeusJiang

Copy link
Copy Markdown
Owner Author

已再次合并最新 main 并解决 Docker 文档冲突;PR 描述已同步。

@ThaddeusJiang ThaddeusJiang marked this pull request as ready for review June 13, 2026 12:31
@ThaddeusJiang ThaddeusJiang merged commit ecd9bb3 into main Jun 13, 2026
6 checks passed
@ThaddeusJiang ThaddeusJiang deleted the feat/npm-frontend-assets branch June 13, 2026 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant